fix toolkit drift and update version#62
Merged
Merged
Conversation
chore(skills): remove openai sidecars
…redictable language The spec's Core Principles table was updated in PR #46 to use 'reduced variance' instead of 'determinism', but two spec footnotes and two README session-state descriptions kept the old term. This aligns all remaining occurrences with the established reduced-variance framing. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
PR #60 removed all skills/*/agents/openai.yaml sidecars but the Codex section of the README still described them. Remove the sentence since the files no longer exist. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
'does not yet support' implied future support was planned. State the current limitation factually without making claims about GitHub's roadmap. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…aming 'Works seamlessly' directly contradicted the following sentence that acknowledged platform variance. Fold the caveat into the bullet and simplify the paragraph to remove the redundant restatement. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
The link was presented without context, implying it was a canonical or project-maintained source. Add a parenthetical so readers know it is external and not maintained by this project. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…table 'No layering' was wrong — CLAUDE.md and .cursorrules do support nested files. The real gap is the absence of a personal-vs-project separation that travels across all projects and tools. Reframe around what is actually missing rather than a capability these tools do have. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
The README presented both distribution models without explaining the relationship. Add a brief callout at the top of the template section so readers understand when each model applies before committing to a fork. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
The platform table incorrectly stated that Copilot Code Review does not support AGENTS.md. Copilot reads AGENTS.md across VS Code, the coding agent, and code review. Remove the false caveat and expand the platform label to reflect full Copilot support. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…nstall The template distribution model is being phased out. Update the callout at the top of the template section to direct new users to the plugin instead of recommending the template fork. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Changed 'consistent, reproducible' to 'consistent and predictable' to match the replacement wording used in all other deterministic → predictable changes in this branch (addressed Copilot review comment). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Patch release for communication drift fixes. Updates version references across spec, README, AGENTS.md, templates, skills, and sample artifacts. Adds CHANGELOG entry and updates spec last_updated date. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
fix: address communication drift between opinionated toolkit claims and facts
Contributor
There was a problem hiding this comment.
Pull request overview
This PR bumps the AI Context Kit session specification to v1.4.3 and updates templates/skills/docs to reduce terminology drift (notably around “deterministic” wording) while removing legacy Codex UI sidecar metadata files (agents/openai.yaml) and correcting related documentation.
Changes:
- Bump spec and stamped template/sample versions from 1.4.2 → 1.4.3 (spec, templates, AGENTS, sample user context).
- Update skill workflows to treat versions older than 1.4.3 as “older” for validation, and stamp generated artifacts with 1.4.3.
- Remove
skills/*/agents/openai.yamlsidecar files and adjust docs/changelog language around provider variance and skill discovery.
Reviewed changes
Copilot reviewed 26 out of 26 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| usercontexts/sample_usercontext.instructions.md | Updates sample user context to spec v1.4.3. |
| templates/usercontext_template.instructions.md | Updates canonical user context template to spec v1.4.3 (including footer stamp). |
| templates/AGENTS_template.md | Updates AGENTS template spec_version stamp to 1.4.3. |
| specs/context_aware_ai_session_spec.md | Bumps spec to v1.4.3 and updates terminology to “consistent and predictable”. |
| docs/spec-rationale.md | Bumps rationale doc to v1.4.3 (frontmatter/title). |
| CHANGELOG.md | Adds 1.4.3 entry describing terminology and documentation fixes. |
| AGENTS.md | Updates spec reference to v1.4.3 in project guidance. |
| skills/create-usercontext-instructions/SKILL.md | Updates generated artifact stamp to spec_version 1.4.3. |
| skills/create-usercontext-instructions/references/quality-checklist.md | Updates checklist to require spec_version 1.4.3. |
| skills/create-project-instructions/SKILL.md | Updates generated AGENTS stamp to spec_version 1.4.3. |
| skills/create-agents-md/SKILL.md | Updates generated AGENTS stamp to spec_version 1.4.3. |
| skills/validate-usercontext-instructions/SKILL.md | Updates validation threshold to warn when older than 1.4.3. |
| skills/validate-project-instructions/SKILL.md | Updates validation threshold to warn when older than 1.4.3. |
| skills/validate-agents-md/SKILL.md | Updates validation threshold to warn when older than 1.4.3. |
| skills/validate-usercontext-instructions/agents/openai.yaml | Deleted (removes legacy OpenAI/Codex UI sidecar). |
| skills/validate-skill/agents/openai.yaml | Deleted (removes legacy OpenAI/Codex UI sidecar). |
| skills/validate-project-instructions/agents/openai.yaml | Deleted (removes legacy OpenAI/Codex UI sidecar). |
| skills/validate-agents-md/agents/openai.yaml | Deleted (removes legacy OpenAI/Codex UI sidecar). |
| skills/restore-checkpoint/agents/openai.yaml | Deleted (removes legacy OpenAI/Codex UI sidecar). |
| skills/repository-drift-control/agents/openai.yaml | Deleted (removes legacy OpenAI/Codex UI sidecar). |
| skills/create-usercontext-instructions/agents/openai.yaml | Deleted (removes legacy OpenAI/Codex UI sidecar). |
| skills/create-skill/agents/openai.yaml | Deleted (removes legacy OpenAI/Codex UI sidecar). |
| skills/create-project-instructions/agents/openai.yaml | Deleted (removes legacy OpenAI/Codex UI sidecar). |
| skills/create-checkpoint/agents/openai.yaml | Deleted (removes legacy OpenAI/Codex UI sidecar). |
| skills/create-agents-md/agents/openai.yaml | Deleted (removes legacy OpenAI/Codex UI sidecar). |
| README.md | Adjusts provider-variance wording and updates notes related to Codex/Copilot behavior. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+4
to
6
| spec_version: 1.4.3 | ||
| created: 2025-10-20 | ||
| last_updated: 2026-05-09 |
Comment on lines
+21
to
+22
| - Replaced all surviving `deterministic` phrasing (spec §4.1 footnote, §10.8 workflow table; README session state section) with `consistent and predictable` — aligns with the reduced-variance language established in v1.4.2 | ||
| - Removed stale `agents/openai.yaml` sidecar claim from README Codex section (sidecars were deleted in v1.4.2 but the description was not updated) |
| - Keep templates and skills aligned with spec `v1.4.2`. | ||
| - Keep templates and skills aligned with spec `v1.4.3`. | ||
| - Maintain the AGENTS-first project-context model. | ||
| - Preserve deterministic behavior and path stability. |
| @@ -397,8 +397,6 @@ claude plugin update ai-context-kit | |||
|
|
|||
| Codex auto-discovers skills from the `.agents/skills/` directory, scanning upward from the current working directory to the repo root. AI Context Kit ships a `.agents/skills/` directory whose entries are symlinks to the canonical `skills/` folders, so repo-local skill discovery and plugin packaging both point at the same source of truth. | |||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.